Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge nci modules #276

Open
wants to merge 15 commits into
base: develop
Choose a base branch
from
Open

Merge nci modules #276

wants to merge 15 commits into from

Conversation

Ariana-B
Copy link
Contributor

Update the NCI Module builder to only create a single module, with all dependencies installed within it. No more 'dea-env' as well as 'dea'.
Environment spec is the same as the 'dea-sandbox' docker image, with eodatasets3, odc-algo, odc-dscache, and odc-io now included in the conda env instead of being installed via pip.

@Ariana-B Ariana-B requested a review from omad November 22, 2023 03:58
@robbibt
Copy link
Member

robbibt commented Nov 24, 2023

Hey @Ariana-B @omad, with the tide modelling stuff, would it be possible to have that global env var actually set when a user runs module load dea? That way it would allow our users to be able to run any of the coastal notebooks/tools without any setup on their end, similar to how it currently works on the Sandbox.

e.g. like this, but set for all our users, not just for tests:
https://github.com/GeoscienceAustralia/dea-sandbox/pull/276/files/b2a32d64e9ef4b6cfbce2261eec8407d330c1ee4..649387e5749d3806d8dd7bcf1d1ae70c306c9f12#diff-5c429f94d7357a48ff2ac30fb36497b6538970ca128e6c9ad158a621f043fb27R35

Copy link

The container image has inefficient files.

Copy link

The container image has inefficient files.

Copy link

github-actions bot commented Jul 1, 2024

The container image has inefficient files.

Copy link
Contributor

@omad omad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants